Create functions for plotting #227
Conversation
gnawin
added
the
question
|
Hi @abelsiqueira @datejada @clizbe, |
| @@ -6,6 +6,9 @@ using DataFrames | |||
| using Graphs | |||
| using HiGHS | |||
| using JuMP | |||
| using Plots | |||
|
|
|||
| export plot | |||
|
|
||
| edge_label_list = [edge_labels[(src(e), dst(e))] for e in edges(graph)] | ||
| g = gplot(graph; edgelabel = edge_label_list) | ||
| display(g) |
There was a problem hiding this comment.
Hi @abelsiqueira ,
The rationale for having this file is given above
In addition, I realized there is no existing function for running the model and deriving the results (apart from the test). For this purpose, also a file in src has been added. This may related to #209. Is creating such a file a desired way to do so?
The line 16-18 is just some early codes for plotting the graph with edge values, which are not working yet.
I'm not sure which you'd like to dump, but feel free to do so as I won't be able to follow it up soon. Thanks.
| show_flow_time_series() | ||
|
|
||
| Plots a time series for a flow, given asset `from_flow`, asset `to_flow`, | ||
| `representative_period`, `set_time_steps`, and solution flow. | ||
| """ | ||
| function show_flow_time_series( | ||
| from_flow, | ||
| to_flow, | ||
| representative_period, | ||
| set_time_steps, | ||
| v_flow, | ||
| ) |
There was a problem hiding this comment.
Some of these names have to be checked against #208
Pull request details
Describe the changes made in this pull request
This is a draft PR.
Aims to create (interactive) functions for the following plots:
In this PR, the first plot has been included, and the second is a WIP.
In addition, I realized there is no existing function for running the model and deriving the results (apart from the test). For this purpose, also a file in src has been added. This may related to #209. Is creating such a file a desired way to do so?
On some technical notes,
show_flow_time_series,plot()did not work, butTulipaEnergyModel.plot()did. Therefore I addedexport plotin TulipaEnergyModel.jl. But it feels weird.@datejada does this resemble more or less what you have in mind? I feel the changes so far deviate from the initial discussion, so it's better to discuss them before I'm caught away.
@abelsiqueira any ideas on the comment regarding using analysis_results.jl and the technical notes?
List of related issues or pull requests
Concerns #191 and possibly to some extent overlaps with #209
Collaboration confirmation
As a contributor I confirm